Skip to content

.deb naming regression fix, plus some random code improvement #157

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jan 13, 2016

Conversation

SethTisue
Copy link
Member

progress on scala/scala-dev#69

this happened when we upgraded sbt-native-packager from 0.6.x
to 1.0.x; the new version calculates this differently
because warnings are a good thing & we want to see them
@SethTisue
Copy link
Member Author

don't merge yet, I need to actually test it on Linux first. (Debian packaging currently doesn't work when run on Mac OS X)

@SethTisue
Copy link
Member Author

ran it on jenkins-worker-ubuntu-publish, show s3Upload::mappings now includes (/home/jenkins/workspace/scala-2.11.x-release-package-unix/target/scala_2.11.7-400_all.deb,scala/2.11.7/scala-2.11.7.deb)

SethTisue added a commit that referenced this pull request Jan 13, 2016
.deb naming regression fix, plus some random code improvement
@SethTisue SethTisue merged commit 8b824c1 into scala:2.11.x Jan 13, 2016
@SethTisue SethTisue deleted the fix-deb-name-regression branch January 13, 2016 16:12
SethTisue added a commit to SethTisue/scala-dist that referenced this pull request Mar 3, 2016
the upgrade turned out to be way too problematic.  even after multiple
PRs addressing regressions, new ones continue to turn up; see
scala/scala-dev#92 for details on the latest
regressions. for past history (including details on regressions), see
these PRs in this repo: scala#159, scala#157, scala#156, scala#155, scala#154, scala#142, plus issue

so what's next after this?

- we could maybe still consider upgrading for 2.11.9, but someone
  would need to thoroughly QA it on all platforms and assure us there
  are no regressions

- or we could restrict the upgrade to 2.12.x and hope for
  partially crowdsourced QA so that regressions would be caught
  during the milestone and release candidate phases.

I lean towards leaving 2.11.x frozen at 0.6.4, at least unless the
upgrade brings concrete benefits to end users (no one has listed
any, to my knowledge).  if this is mainly just dogfooding, then
2.12.x is a better context for that.
SethTisue added a commit to SethTisue/scala-dist that referenced this pull request Mar 3, 2016
this restores the 2.11.7 status quo for the 2.11.8 release.

the upgrade turned out to be way too problematic.  even after multiple
PRs addressing regressions, new ones continue to turn up; see
scala/scala-dev#92 for details on the latest
regressions. for past history (including details on regressions), see
these PRs in this repo: scala#159, scala#157, scala#156, scala#155, scala#154, scala#142, plus issue

so what's next after this?

- we could maybe still consider upgrading for 2.11.9, but someone
  would need to thoroughly QA it on all platforms and assure us there
  are no regressions

- or we could restrict the upgrade to 2.12.x and hope for
  partially crowdsourced QA so that regressions would be caught
  during the milestone and release candidate phases.

I lean towards leaving 2.11.x frozen at 0.6.4, at least unless the
upgrade brings concrete benefits to end users (no one has listed
any, to my knowledge).  if this is mainly just dogfooding, then
2.12.x is a better context for that.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant